Skip to content

updated precompiled headers based on ClangBuildAnalyzer#4775

Merged
danmar merged 4 commits intocppcheck-opensource:mainfrom
firewave:precompile
Feb 8, 2023
Merged

updated precompiled headers based on ClangBuildAnalyzer#4775
danmar merged 4 commits intocppcheck-opensource:mainfrom
firewave:precompile

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Feb 7, 2023

First I ran ClangBuildAnalyzer without the precompiled headers enabled to determine the heavy ones. Then I ran it again with the precompiled headers enabled to see if there were more beyond the top 10 ones.

In case of lib there is no change - mainly because it's just different headers which lead up to the same stuff being included. For test there's a very slight improvement only.

For cppcheck-gui there's obviously a big improvement since we had none so far:

before:

Compilation (86 times):
  Parsing (frontend):          392.1 s
  Codegen & opts (backend):    119.2 s

after:

Compilation (87 times):
  Parsing (frontend):           96.7 s
  Codegen & opts (backend):    136.2 s

@danmar danmar merged commit 66758d6 into cppcheck-opensource:main Feb 8, 2023
@firewave firewave deleted the precompile branch February 8, 2023 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants